Skip to content

feat: pretty printing#5

Open
onefloid wants to merge 6 commits into
mainfrom
feat/pretty-printing
Open

feat: pretty printing#5
onefloid wants to merge 6 commits into
mainfrom
feat/pretty-printing

Conversation

@onefloid

@onefloid onefloid commented Jun 8, 2025

Copy link
Copy Markdown
Owner

This PR contains:

  • Refactoring of exists commands
  • Pretty prints if --output--raw is not set

Open tasks before Merge

  • Update readme
  • Update all tests

Issues
Closes #4

@onefloid onefloid marked this pull request as ready for review September 15, 2025 10:24
claude added 2 commits July 5, 2026 19:47
- utils/generic.py: narrow the broad except in execute_exists() so it
  no longer swallows typer.Exit. Since typer.Exit subclasses
  RuntimeError/Exception, resolve_database()'s print_error_and_exit()
  call (e.g. for an unknown --database name) was being re-caught and
  produced a spurious extra "Exit:" line after the real error message.
- utils/generic.py: replace the attribute_name[:-1].capitalize()
  depluralization with an explicit SINGULAR_NAMES mapping, since
  stripping one character mangled "processes" into "Processe" instead
  of "Process".
- utils/generic.py: add the missing `# pylint: disable=redefined-builtin`
  on `from rich import print`, matching project convention and fixing
  the CI pylint failure.
- main.py, commands/process.py: switch `from typing_extensions import
  Annotated` to the stdlib `from typing import Annotated`, matching
  the other command modules. Newer typer releases (needed after
  loosening the typer version constraint on main) no longer pull in
  typing_extensions transitively, so the old import broke on a fresh
  install.
- tests: update test_cmd_cubes.py, test_cmd_subset.py and
  test_cmd_view.py assertions for the new pretty-printed output and
  add --output-raw coverage, mirroring the pattern already used in
  test_cmd_dimension.py. Closes the "Update all tests" item from the
  PR description.
- README.md: document the new --output-raw flag. Closes the "Update
  readme" item from the PR description.

Verified: full mocked test suite (19/19) and pylint (10.00/10) pass
on a fresh install of the branch's dependencies.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Following The DRY Principle

2 participants